-
Notifications
You must be signed in to change notification settings - Fork 145
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Streaming Timeseries #838
Streaming Timeseries #838
Conversation
Hello @matteobachetti! Thanks for updating this PR. We checked the lines you've touched for PEP 8 issues, and found:
Comment last updated at 2024-09-09 08:58:27 UTC |
Accept file names instead of hdulist Allow for shorter exposure cuts than GTIs
Codecov ReportAttention: Patch coverage is
Additional details and impacted files@@ Coverage Diff @@
## main #838 +/- ##
===========================================
- Coverage 96.53% 79.20% -17.33%
===========================================
Files 48 48
Lines 9257 9336 +79
===========================================
- Hits 8936 7395 -1541
- Misses 321 1941 +1620 ☔ View full report in Codecov by Sentry. |
I converted to draft. There is some work to be done |
These new methods serve the purpose of streaming time series objects by segment. This is a prerequisite for the parallelization of the
Averaged*spectrum
calculation.